-
Notifications
You must be signed in to change notification settings - Fork 309
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Audio: multiband_drc: multiband drc code re-organize and split #8526
Conversation
btian1
commented
Nov 27, 2023
•
edited
Loading
edited
- Move header file from include path to module inside folder.
- Move one inline function to header file and remove one inline function.
- split multiband DRC with ipc3 and ipc4 specific code.
Move header file from include path to module inside folder. Signed-off-by: Baofeng Tian <[email protected]>
7a0c945
to
0a2cfd0
Compare
Move one inlined function to header file, for the second functions, it was called 4 times in current module and not time critical, no need inline. Signed-off-by: Baofeng Tian <[email protected]>
0f9f649
to
d2a8682
Compare
@btian1 any reason why draft ? |
just waiting for CI results, now opened. |
zephyr/CMakeLists.txt
Outdated
elseif(CONFIG_IPC_MAJOR_4) | ||
zephyr_library_sources_ifdef(CONFIG_COMP_MULTIBAND_DRC | ||
${SOF_AUDIO_PATH}/multiband_drc/multiband_drc.c | ||
${SOF_AUDIO_PATH}/multiband_drc/multiband_drc_generic.c |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No copy/paste, see #8514 (comment)
d2a8682
to
a919009
Compare
Split out ipc3 and ipc4 specific code from mulitband_drc.c. Signed-off-by: Baofeng Tian <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, looks good @btian1 !
SOFCI TEST |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM